Skip to content

🔁 Modernized index landing page#2413

Open
JFWooten4 wants to merge 11 commits into
stellar:mainfrom
JFWooten4:723-pt-10
Open

🔁 Modernized index landing page#2413
JFWooten4 wants to merge 11 commits into
stellar:mainfrom
JFWooten4:723-pt-10

Conversation

@JFWooten4

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 29, 2026 07:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Stellar Developer Docs landing page content and layout to a more modern introduction, including an embedded explainer video.

Changes:

  • Replace the introductory copy with updated messaging.
  • Remove the unused PathfindingTable import and add a YouTube embed to the homepage.
  • Adjust the “developer docs” description copy (including “on- and off-ramps” wording).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/pages/index.mdx Outdated
Comment thread src/pages/index.mdx Outdated
Comment thread src/pages/index.mdx Outdated
@JFWooten4

Copy link
Copy Markdown
Contributor Author
comparison

@ElliotFriend ElliotFriend left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this — there are three things in here I definitely want, and two I'm going to decline. Let me be specific so you can trim this to a fast merge.

Keep (please reduce the PR to these):

  1. loading="lazy" on the YouTube component — nice catch, and it now benefits every meeting-note page using the component.
  2. Removing the unused PathfindingTable import — confirmed dead on main.
  3. The "on- and off-ramps" hyphenation fix — correct suspended hyphen.

Declining:

  • The video embed: homepage content is curated with our DevRel/comms folks, and we're not adding a featured video right now. If you think one belongs there, an issue is the right venue so the right people can weigh in on which video and where.
  • The intro prose rewrite: this is a voice call. The current phrasing ("will teach you...") is the tone we want on the landing page, and additions like "for everyone" read more marketing than docs. Keeping the original.

Strip it down to the three keepers and I'll merge it quickly. Thanks!

@JFWooten4

Copy link
Copy Markdown
Contributor Author

Changes made, and I see what you're getting at with the team review. I have only seen a small group of accounts commenting on here for more design-focused issues I opened before. I appreciate and see that there can be more discussion with relevant parties.

Namely, it was my interpretation of the Stellar deprecation that it came from a "higher-up" decision authority. Obviously, there wasn't much discussion of the transition on here. I would appreciate having more open development communications and planning debates here as suggested.

Thanks to your comment on #2571 and especially private communications from Bri in helping to form the views on team collaboration and decisions

The homepage looks good, and I see what you're getting at with the informative tone. But I do dissent because I'd much rather use the active voice so that sentences flow together cohesively. I've seen that infinitive phrasing throughout a lot of the docs and, while I can appreciate its use, I try to introduce a subject and a noun object, as I think we should to prevent confusion like #2398 (comment) (where a cleaner read would've been "Validators [did something].")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants